Skip to content

Update CONTRIBUTING.md#7570

Merged
Luke9389 merged 1 commit into
mainfrom
mallenexpensify-patch-1
Feb 4, 2022
Merged

Update CONTRIBUTING.md#7570
Luke9389 merged 1 commit into
mainfrom
mallenexpensify-patch-1

Conversation

@mallenexpensify

@mallenexpensify mallenexpensify commented Feb 4, 2022

Copy link
Copy Markdown
Contributor

Removed "If you are unable to build to iOS/OSX due to using a Windows machine, please let the reviewers know so they can double check that platform themselves." because we added a way to test mac on windows, it's in/around line 72

Details

Removed "If you are unable to build to iOS/OSX due to using a Windows machine, please let the reviewers know so they can double check that platform themselves." because we added a way to test mac on windows, it's in/around line 72

Fixed Issues

$ GH_LINK

Tests

Check CONTRIBUTING.md to ensure "If you are unable to build to iOS/OSX due to using a Windows machine, please let the reviewers know so they can double check that platform themselves." is removed

  • Verify that no errors appear in the JS console

QA Steps

Check CONTRIBUTING.md to ensure "If you are unable to build to iOS/OSX due to using a Windows machine, please let the reviewers know so they can double check that platform themselves." is removed

  • Verify that no errors appear in the JS console

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

Mobile Web

Desktop

iOS

Android

Removed "If you are unable to build to iOS/OSX due to using a Windows machine, please let the reviewers know so they can double check that platform themselves."  because we added a way to test mac on windows, it's in/around line 72
@mallenexpensify mallenexpensify requested a review from a team as a code owner February 4, 2022 19:37
@MelvinBot MelvinBot requested review from Luke9389 and removed request for a team February 4, 2022 19:37
@mallenexpensify mallenexpensify self-assigned this Feb 4, 2022

@Luke9389 Luke9389 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@Luke9389

Luke9389 commented Feb 4, 2022

Copy link
Copy Markdown
Contributor

I'll merge once travis is done. 👍

@Luke9389

Luke9389 commented Feb 4, 2022

Copy link
Copy Markdown
Contributor

Man, iOS tests are taking a while! (almost 45 mins now)...

@Luke9389 Luke9389 merged commit 0d349d2 into main Feb 4, 2022
@Luke9389 Luke9389 deleted the mallenexpensify-patch-1 branch February 4, 2022 21:03
@OSBotify

OSBotify commented Feb 4, 2022

Copy link
Copy Markdown
Contributor

✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@OSBotify

OSBotify commented Feb 7, 2022

Copy link
Copy Markdown
Contributor

🚀 Deployed to staging by @Luke9389 in version: 1.1.37-0 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

@OSBotify

OSBotify commented Feb 9, 2022

Copy link
Copy Markdown
Contributor

🚀 Deployed to production by @roryabraham in version: 1.1.37-2 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 failure ❌
🕸 web 🕸 success ✅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants